Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix uninitialized tlsConfig in influxdb plugin #10899

Merged
merged 1 commit into from
Mar 19, 2021
Merged

fix uninitialized tlsConfig in influxdb plugin #10899

merged 1 commit into from
Mar 19, 2021

Conversation

jleinfors
Copy link
Contributor

fix for panic when configuring influxdb with tls but without pem_bundle or pem_json
crash log: https://gist.github.com/jleinfors/425c105f0146ba89ccb934f69ddce5d0

@hashicorp-cla
Copy link

hashicorp-cla commented Feb 11, 2021 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@vercel vercel bot temporarily deployed to Preview – vault-storybook February 11, 2021 12:57 Inactive
@vercel vercel bot temporarily deployed to Preview – vault February 11, 2021 12:57 Inactive
@jleinfors jleinfors force-pushed the johlei/influxdb-tls-fix branch from 45eae1a to 4b77c40 Compare February 11, 2021 13:13
@vercel vercel bot temporarily deployed to Preview – vault-storybook February 11, 2021 13:13 Inactive
@vercel vercel bot temporarily deployed to Preview – vault February 11, 2021 13:13 Inactive
@jleinfors jleinfors force-pushed the johlei/influxdb-tls-fix branch from 4b77c40 to 388757d Compare February 11, 2021 18:55
@vercel vercel bot temporarily deployed to Preview – vault February 11, 2021 18:55 Inactive
@vercel vercel bot temporarily deployed to Preview – vault-storybook February 11, 2021 18:55 Inactive
@jleinfors jleinfors requested a review from kalafut February 15, 2021 13:38
@kalafut kalafut requested a review from a team March 19, 2021 03:10
kalafut
kalafut approved these changes Mar 19, 2021
View reviewed changes
Copy link
Contributor

@kalafut kalafut left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kalafut kalafut added this to the 1.7.1 milestone Mar 19, 2021
@kalafut kalafut merged commit e715f63 into hashicorp:master Mar 19, 2021
@kalafut kalafut modified the milestones: 1.7.1, 1.7 Mar 19, 2021
kalafut pushed a commit that referenced this pull request Mar 19, 2021
@jleinfors
fix uninitialized tlsConfig in influxdb plugin (#10899)
9ebe525
@kalafut kalafut mentioned this pull request Mar 19, 2021
Merged
kalafut pushed a commit that referenced this pull request Mar 19, 2021
@jleinfors
fix uninitialized tlsConfig in influxdb plugin (#10899) (#11148)
f607096 
Co-authored-by: Johan Leinfors <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kalafut kalafut kalafut approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.7
Development

Successfully merging this pull request may close these issues.
3 participants
@jleinfors @hashicorp-cla @kalafut